Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make some adjustments to mx_UserPill and mx_RoomPill #4597

Merged
merged 2 commits into from
Jul 19, 2017

Conversation

lukebarnard1
Copy link
Contributor

@lukebarnard1 lukebarnard1 commented Jul 17, 2017

So that they can appear correctly in the body of a text message.

2017-07-19-110426_777x100_scrot
2017-07-19-094706_784x426_scrot

So that they can appear correctly in the body of a text message.
@dbkr dbkr assigned lukebarnard1 and unassigned dbkr Jul 17, 2017
@lukebarnard1
Copy link
Contributor Author

lukebarnard1 commented Jul 19, 2017

I'm going to have to change this so that it matches the design - user pills should not have a background colour in the timeline.

@lukebarnard1 lukebarnard1 assigned dbkr and unassigned lukebarnard1 Jul 19, 2017
@dbkr
Copy link
Member

dbkr commented Jul 19, 2017

yep, seems reasonable

@lukebarnard1
Copy link
Contributor Author

Given this is just a CSS change, I'm assuming the failing react-sdk test is unrelated (https://travis-ci.org/vector-im/riot-web/jobs/255197083 <-- @richvdh)

@lukebarnard1 lukebarnard1 merged commit f7d282f into develop Jul 19, 2017
@t3chguy t3chguy deleted the luke/css-matrixto-timeline-pills branch May 12, 2022 08:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants